-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: Fix node 15 #809
test: Fix node 15 #809
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 85bf40d:
|
@kentcdodds Could you elaborate why we have the separate
|
@eps1lon it looks like @kentcdodds wrote in #808 (comment) that we can remove the separate |
I thought we still wanted to test Node 15, but using |
I can do this all day 😬 |
Codecov Report
@@ Coverage Diff @@
## master #809 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 4 4
Lines 117 117
Branches 16 16
=========================================
Hits 117 117 Continue to review full report at Codecov.
|
Looks ready to merge for me |
🎉 This PR is included in version 11.1.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
What:
Explicitly add node 15 to the matrix and fix compat issues.
Why:
We declare that we support node >= 10 so we should test accordingly. Now that node 15 is released we can declare it in the matrix.
How:
Follow advise in npm/cli#1935 until npm fixes its CI detection.
Checklist:
[ ]Documentation added to thedocs site
[ ]Typescript definitions updated